Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

docs(ngMessageExp): Better description #15486

Closed
wants to merge 2 commits into from

Conversation

spaceribs
Copy link
Contributor

@spaceribs spaceribs commented Dec 9, 2016

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

docs update

What is the current behavior? (You can also link to an open issue here)

ngMessageExp has the same description as ngMessage

What is the new behavior (if this is a feature change)?

Updated description

Does this PR introduce a breaking change?

No

Please check if the PR fulfills these requirements

Other information:

Old description was copied over from ngMessage.

Old description was copied over from ngMessage.
* For `ngMessageExp` to operate, a parent `ngMessages` directive on a parent DOM element
* must be situated since it determines which messages are visible based on the state
* of the provided key/value map that `ngMessages` listens on.
* `ngMessageExp` is the same as {@link module:ngMessage `ngMessage`} but instead
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be directive:ngMessage or something (instead of module:ngMessage)?

* must be situated since it determines which messages are visible based on the state
* of the provided key/value map that `ngMessages` listens on.
* `ngMessageExp` is the same as {@link module:ngMessage `ngMessage`} but instead
* of a DOM attribute, accepts an expression to be evaluated for the message key.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"DOM attribute" isn't correct. How about "static value"?

link needs to point to the ngMessage directive, and DOM attribute isn't clear enough
@spaceribs
Copy link
Contributor Author

@gkalpak updated based on feedback, thanks

gkalpak pushed a commit that referenced this pull request Dec 11, 2016
@gkalpak gkalpak closed this in 29f3bf4 Dec 11, 2016
gkalpak pushed a commit that referenced this pull request Dec 11, 2016
ellimist pushed a commit to ellimist/angular.js that referenced this pull request Mar 15, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants